-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Хмелёв Дима #51
base: master
Are you sure you want to change the base?
Хмелёв Дима #51
Conversation
🍅 Не пройден линтинг или базовые тесты |
🍅 Не пройден линтинг или базовые тесты |
🍅 Не пройден линтинг или базовые тесты |
🍅 Не пройден линтинг или базовые тесты |
🍅 Не пройден линтинг или базовые тесты |
🍏 Пройден линтинг и базовые тесты |
border: 5px double; | ||
} | ||
|
||
.headerOne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first-header
height: 75%; | ||
} | ||
|
||
.headerTwo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
second-header
Не используй camelCase для именования классов |
width: 15%; | ||
} | ||
|
||
.headerName |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не очень удачное имя класса
font-family: 'Lobster', cursive; | ||
} | ||
|
||
.first |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Первое что?
width: 40%; | ||
} | ||
|
||
.second, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Второе, третье, четвертое что? :)
🍏 Пройден линтинг и базовые тесты |
.blackRect | ||
{ | ||
background-color: black; | ||
width: 20px !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Попробуй обойтись без !important
background-color: black; | ||
} | ||
|
||
.see |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Опять-таки не очень понятное имя
width: 100%; | ||
} | ||
|
||
.ogl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ogl? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
черновой вариант , забыл исправить. Классов чет многа
width: 300px; | ||
height: 216px; | ||
border: 2px double; | ||
float: right; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В этом задании нужно обойтись без плавающих элементов
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А как тогда выравнивать картинки относительно блоков? Или вставлять картинку в середину текста?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Творить магию :)
|
||
.page | ||
{ | ||
float: right; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
То же замечание, что и выше
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Либо что-то пошло не так, либо я все еще вижу float
font-size: 10px; | ||
} | ||
|
||
.pictureMainFirst |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
поясни, пожалуйста, название класса?
<title>Прибамбасск</title> | ||
<meta charset="UTF-8"> | ||
<link href="index.css" rel="stylesheet"> | ||
<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Будет хорошо, если попробуешь подключить шрифты не только с google fonts, но и самостоятельно
<div class="world"> | ||
<img src="img/Мегаторг.png" class="mega" alt="Мегаторг" title="Магазин"> | ||
все товары<br> | ||
<div class="big-world">Мира</div><br> | ||
под одной крышей | ||
</div> | ||
</figure> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
почему ты переделал на div?
🍅 |
🍅 Не пройден линтинг или базовые тесты |
🍅 Не пройден линтинг или базовые тесты |
🍏 Пройден линтинг и базовые тесты |
@@ -83,7 +83,7 @@ | |||
благодарность работникам порта. Среди главных | |||
Факторов, определивших успех Прибамбасска, мэр | |||
назвал удобное прибрежное расположение города — | |||
рядом с Великим морем, а так же политику всячиски | |||
рядом с Великим морем, а так же политику всячиски |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
всячески
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ахахахаха , я с картинки программой текст получал)
🍅 |
🍏 Пройден линтинг и базовые тесты |
Почему я до сих пор вижу плавающий элемент и font-size small? :( |
🍏 Пройден линтинг и базовые тесты |
🚀 |
Тематически очень хорошо, реализация пока что не особо.
|
выпуск | ||
</div> | ||
<div class="header-name"> | ||
Вестник Прибамбасска<br> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем этот br?
<img src="img/Море.png" alt="Море" title="Море"> | ||
</p> | ||
<main> | ||
<table> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Таблица тут вобще для раскладки не нужна.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ну тогда сложно padding-ми все выравнивать , так как float то же нельзя.Я первый раз все картинки флоатами выравнивал. Сказали нельзя. Я могу все inline дивами , все сделать. Но внутри все равно придется паддингами двигать
<h1> | ||
Доходы растут | ||
</h1> | ||
<p class="main-picture"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p – это paragraph, использовать его в качестве обёртки для картинки не лучшая идея, существует более специализированный тег именно для этой цели
<img src="img/Тотем.png" class="totem" alt="ТотемыРеклама" title="Реклама"> | ||
</td> | ||
<td> | ||
<div class="table-of-contents"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Оглавление можно сверстать как-нибудь поизящней, списком например
.header | ||
{ | ||
position: relative; | ||
overflow: hidden; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем overflow: hidden?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
черновой вариант , забыл удалить
|
||
.first-header | ||
{ | ||
height: 75%; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
75% чего?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
75% всего header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Дело в том что у header-то высота не задана и поэтому это правило не работает.
https://yadi.sk/i/eY2uheF4yQCxP этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь :) Не знаю даже что посоветовать, давайте попробуем сделать хотя бы так, чтобы элементы в нём как-то более равномерно пространство занимали чтоли. Можно ещё купон выделить с помошью dashed бордера |
То что надпись "ежедневно" прыгает туда-сюда в зависимости от ширины экрана это баг или фича? https://yadi.sk/i/Jk4snAvHyQDr4 |
"этот блок такой странный, что я не могу понять, он так и должен выглядеть или развалился весь" |
Окей, этот вопрос снимается. |
Но остальное в силе. Забыл помидор. 🍅 |
Посмотреть решение